ConnectionManager implementation and status of IoT Cloud connection #23
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- small tweaks to ConnectionManager (verbose level) - needs more work
…ing brokerAddress) as an overloaded method initialising the connection object and still relying upon begin(Client& net, String brokerAddress). This allows us to maintain backwards compatibility as well as allowing users to implement their own means of connection.
- fine-tuned CHECK_INTERVAL_ values (might use more work)
- update() checks against connection not being NULL
|
[ UPDATE: fixed ] |
- adjusted default output message verbosity - refactored ArduinoIoTCloud - added updated example
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The changes applied to ArduinoIoTCloud include the implementation of a generic abstract class to handle connection/disconnection/reconnection.
An overloaded method has been added to accept a ConnectionManager* object and defer tests of current network connection status to it during ArduinoCloud.update();
Also the state machine implemented for connection states has proven to be very reliably even on boards with old firmware which struggle to obtain network time.
For now only the WiFiConnectionManager class is implemented.
Work on the GSMConnectionManager will follow.